Nodemon refactor & JSON config #200
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
You can add ignore files from a nodemon.json file { ignore: [ files, ]
}
|
Yay! I'm glad to see that the talk was useful to others. |
|
@mwbrooks I actually missed the talk, and am eagerly awaiting the video to be released, but I got quite a bit from your slides and from various projects have quite a bit of familiarity already. Still, definitely useful stuff. |
Includes simple passing tests, the exec.js has not been tested in detail yet.
Still lots to do, but taking a very code, test, code, test, test, test, code, test cycle.
This was referenced Aug 5, 2013
Making it so lib/cli/ can just spew out options, and make it easier to test against too.
|
A hack that works for me at the moment is: That is now if you don't mind the double output in the terminal. This should work on any unix based system. |
Also config static module is used by the parts of nodemon to run the thing. Ignore rules, on the command line also work - a nice new feature.
- Loads $HOME/nodemon.json - Then ./nodemon.json - Then overwrites with cli (or module) settings - -V gives detail of what's going on inside nodemon - -q shuts nodemon up entirely - Cleaner config loading breakup
- Events documented - Require working nicely - If nodemon is required, nodemon message are emitted rather than console.log'ged
|
Require tests failing in linux, but passing on Mac. Yay :( will fix this w/e. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
To start experimenting with this build:
npm install -g nodemon@devand please leave feedback & file issuesThis is the start of a complete restructuring of nodemon's source code, since currently it's a massive one script which can be difficult to test (and frankly the tests are limited right now) and difficult to contribute to.
So I'm breaking all the functionality down to small modules.
Key features of the new work:
nodemon.jsonwith overriding defaults kept in~/nodemon.json. The JSON file will allow for all command line options to be stored in JSONThe whole code base is getting gutted, but I'm wary of making sure that the actual code gets carried across (because I know there's been a lot of fixes for Windows and edge cases).
Right now, the code is not in a good state, I'm still the file up, but I'm up for contributions, particularly to test modules.
I'm using @mwbrooks' and @filmaj's slides from their experience building the Cordova & PhoneGap CLI as inspiration: http://michaelbrooks.ca/deck/2013-node-brigade/
Addition feature checklist